Clean up GoReleaser release publishing config by bomly-guy · Pull Request #174 · bomly-dev/bomly-cli

bomly-guy · 2026-06-17T08:59:47Z

Summary

remove the workflow-generated RELEASE_NOTES.md file and --release-notes argument
keep GoReleaser's native release.header as the single release-body intro
expand the header to explicitly list generated artifact classes and package-manager publishing outputs
remove explicit same-repo pull_request.base blocks for Homebrew and Scoop so GoReleaser no longer tries GitHub's fork-sync API for Bomly-owned package repos
keep the WinGet cross-repo base untouched, since that PR targets microsoft/winget-pkgs

This combines the release-note cleanup from #174 with the package repo sync cleanup from #173.

Validation

ruby -e 'require "yaml"; YAML.load_file(".github/workflows/release.yml"); puts "release.yml OK"'
/Users/ahmed/go/bin/goreleaser check
git diff --check
make test
/Users/ahmed/go/bin/goreleaser release --snapshot --clean --skip=publish

Summary by CodeRabbit

Chores
- Updated release workflow and build configuration to enhance GitHub release notes with improved clarity and more detailed information about included artifacts and supported platforms.

github-actions · 2026-06-17T09:01:24Z

Bomly Diff Summary

Compared 5a1184c2c9861b7ffbad24037fb33b225c7fa8f5 to 415b117d26ca4670517644ad5a9ab688bb108d66.

Overview

Status	Manifests	Dependencies	Findings	Duration
✅ Pass	+0 / ~0 / -0	+0 / ~0 / -0	0 introduced / 0 persisted / 0 resolved	69114ms

Dependency Changes

✅ No dependency changes.

Vulnerabilities

✅ No vulnerability changes.

License Changes

✅ No license changes.

Project Posture

✅ No project posture changes (or --matchers +scorecard was not selected).

Policy Findings

✅ No policy differences were identified.

coderabbitai · 2026-06-17T09:31:22Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 01798186-5f09-493e-99d7-36988d2f77af

📥 Commits

Reviewing files that changed from the base of the PR and between 7f8e484 and 8432773.

📒 Files selected for processing (2)

.github/workflows/release.yml
.goreleaser.yaml

📝 Walkthrough

Walkthrough

The release workflow removes the step that generates RELEASE_NOTES.md and the corresponding --release-notes=... argument from the GoReleaser invocation. The GoReleaser config replaces the draft release header with a detailed artifact list and removes pull_request.base overrides from the homebrew_casks and scoops configurations.

Changes

Release Workflow and GoReleaser Config

Layer / File(s)	Summary
Remove RELEASE_NOTES.md generation and `--release-notes` arg `.github/workflows/release.yml`	Drops the workflow step that populated `$RUNNER_TEMP/RELEASE_NOTES.md` and simplifies the GoReleaser `args` from `release --clean --release-notes=...` to `release --clean`.
Expand draft release header and remove `pull_request.base` overrides `.goreleaser.yaml`	Replaces the single-sentence draft release header with a structured, multi-line artifact listing (archives, `bomly-lite` archives, Linux packages, package-manager manifests, `SHA256SUMS`, attestations note). Removes `pull_request.base` from `homebrew_casks` and `scoops` blocks, retaining only `enabled` and `draft`.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Possibly related PRs

bomly-dev/bomly-cli#171: Added RELEASE_NOTES.md generation and the --release-notes=RELEASE_NOTES.md GoReleaser arg that this PR removes.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main changes: consolidating release notes generation and simplifying GoReleaser's release publishing configuration by removing redundant workflow steps and updating the header content.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch codex/use-goreleaser-release-header

Warning

Review ran into problems

🔥 Problems

Stopped waiting for pipeline failures after 30000ms. One of your pipelines takes longer than our 30000ms fetch window to run, so review may not consider pipeline-failure results for inline comments if any failures occurred after the fetch window. Increase the timeout if you want to wait longer or run a @coderabbit review after the pipeline has finished.

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

… package formats

Use GoReleaser header for release notes

c219696

Avoid fork sync for Bomly package repos

8432773

bomly-guy changed the title ~~Use GoReleaser header for release notes~~ Clean up GoReleaser release publishing config Jun 17, 2026

bomly-guy mentioned this pull request Jun 17, 2026

Avoid fork sync for Bomly package repos #173

Closed

bomly-guy added 3 commits June 17, 2026 02:45

chore: update homepage and descriptions for Bomly CLI across multiple…

7a87c5e

… package formats

chore: update homepage and descriptions for Bomly CLI across multiple…

857622a

… package formats

Merge branch 'main' into codex/use-goreleaser-release-header

415b117

bomly-guy enabled auto-merge (squash) June 17, 2026 09:47

bomly-guy disabled auto-merge June 17, 2026 09:47

bomly-guy merged commit fb87325 into main Jun 17, 2026
11 checks passed

bomly-guy deleted the codex/use-goreleaser-release-header branch June 17, 2026 09:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Clean up GoReleaser release publishing config#174

Clean up GoReleaser release publishing config#174
bomly-guy merged 5 commits into
mainfrom
codex/use-goreleaser-release-header

bomly-guy commented Jun 17, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

github-actions Bot commented Jun 17, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Jun 17, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Review ran into problems

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

bomly-guy commented Jun 17, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Summary by CodeRabbit

Uh oh!

github-actions Bot commented Jun 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Bomly Diff Summary

Overview

Dependency Changes

Vulnerabilities

License Changes

Project Posture

Policy Findings

Uh oh!

coderabbitai Bot commented Jun 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Review ran into problems

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

bomly-guy commented Jun 17, 2026 •

edited by coderabbitai Bot

Loading

github-actions Bot commented Jun 17, 2026 •

edited

Loading

coderabbitai Bot commented Jun 17, 2026 •

edited

Loading